Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Data for detailed booking and booking transformed for forecast rows #579

Merged
merged 7 commits into from
Jan 10, 2025

Conversation

MariaBonde
Copy link
Contributor

@MariaBonde MariaBonde commented Jan 10, 2025

In the forecast, everything is displayed per month, but the data is the same as staffing. While the forecast-data itself comes from the backend and will be transformed there, the data used for the infopills and the hovered state where we show which project the hours derive from must be transformed as well. We decided to do this in the frontend, and I have made a class dedicated to transforming the data, as well as a converter, converting week numbers into months, with data on the distribution of months if a week happens to be in two months.
I was hoping to be able to reuse some of the files that were used in staffing, but because of the weektypes used everywhere I had to create copys where there are not many differences, mostly the word week is replaced by the word month and the data is transformed as needed.
Skjermbilde 2025-01-10 kl  09 02 39
The data is still only testdata, and so the bookingmodel and detailedbooking does not necessarily correspond with each other, so the values do not make so much sense.

@MariaBonde MariaBonde linked an issue Jan 10, 2025 that may be closed by this pull request
@MariaBonde MariaBonde marked this pull request as ready for review January 10, 2025 08:54
frontend/src/api-types.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@idamand idamand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👯

@MariaBonde MariaBonde merged commit 3da9690 into main Jan 10, 2025
3 checks passed
@MariaBonde MariaBonde deleted the 574-bookedhourspermonth branch January 10, 2025 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bookedHoursPerMonth
2 participants